Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore HTML/XML comment blocks #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Ignore HTML/XML comment blocks #17

wants to merge 1 commit into from

Conversation

philbooth
Copy link

Hey Ciaran.

I made this change locally after experiencing the same problem as discussed in issue #12.

Obviously that thread is a year old and you made a strong argument against this change there, so I'm happy if you want to reject it. Just thought I should give you the opportunity to speak for yourself, rather than assuming rejection as a fait accompli! :)

@ciaranm
Copy link
Owner

ciaranm commented Jun 2, 2013

I'm still sceptical that HTML comments are commonly formatted in a way that screws up indenting. And even if they are, should they be?

@philbooth
Copy link
Author

...quite possibly not!

I only made the change so I could use vim on a legacy codebase in my new job. This pull request was made more in the interests of being a responsible open-source citizen (giving back) than because I feel strongly that the change should make it back into your repo. If you're against it, that's cool with me. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants